Skip to content

Conversation

OneZero-Y
Copy link
Contributor

What type of PR is this?:
Add unit tests

Which issue(s) this PR fixes:

Part of #266

1. Run All Unit Tests

  • Runs all unit tests in the Rust library
make test-rust

2. Run Specific Module Tests

make test-rust-module MODULE=<module_name>

  • Runs tests for a specific module or test function
  • Requires MODULE parameter

Examples:

  • Test entire module
make test-rust-module MODULE=ffi::classify_test
  • Test specific function
make test-rust-module MODULE=ffi::classify_test::test_classify_modernbert_token_entity
  • Test core similarity module
make test-rust-module MODULE=core::similarity_test

Copy link

github-actions bot commented Sep 30, 2025

👥 vLLM Semantic Team Notification

The following members have been identified for the changed files in this PR and have been automatically assigned:

📁 candle-binding

Owners: @rootfs
Files changed:

  • candle-binding/src/classifiers/lora/intent_lora_test.rs
  • candle-binding/src/classifiers/lora/pii_lora_test.rs
  • candle-binding/src/classifiers/lora/security_lora_test.rs
  • candle-binding/src/classifiers/lora/token_lora_test.rs
  • candle-binding/src/classifiers/traditional/batch_processor_test.rs
  • candle-binding/src/classifiers/traditional/modernbert_classifier_test.rs
  • candle-binding/src/classifiers/unified_test.rs
  • candle-binding/src/core/config_loader_test.rs
  • candle-binding/src/core/unified_error_test.rs
  • candle-binding/src/ffi/classify_test.rs
  • candle-binding/src/ffi/memory_safety_test.rs
  • candle-binding/src/model_architectures/lora/bert_lora_test.rs
  • candle-binding/src/model_architectures/model_factory_test.rs
  • candle-binding/src/model_architectures/routing_test.rs
  • candle-binding/src/model_architectures/traditional/base_model_test.rs
  • candle-binding/src/model_architectures/traditional/bert_test.rs
  • candle-binding/src/model_architectures/traditional/modernbert_test.rs
  • candle-binding/src/model_architectures/unified_interface_test.rs
  • candle-binding/src/test_fixtures.rs
  • candle-binding/Cargo.toml
  • candle-binding/src/classifiers/lora/mod.rs
  • candle-binding/src/classifiers/mod.rs
  • candle-binding/src/classifiers/traditional/mod.rs
  • candle-binding/src/core/mod.rs
  • candle-binding/src/ffi/mod.rs
  • candle-binding/src/lib.rs
  • candle-binding/src/model_architectures/config.rs
  • candle-binding/src/model_architectures/lora/mod.rs
  • candle-binding/src/model_architectures/mod.rs
  • candle-binding/src/model_architectures/traditional/mod.rs
  • candle-binding/src/utils/memory.rs

📁 tools

Owners: @yuluo-yx, @rootfs, @Xunzhuo
Files changed:

  • tools/make/build-run-test.mk
  • tools/make/common.mk
  • tools/make/rust.mk

vLLM

🎉 Thanks for your contributions!

This comment was automatically generated based on the OWNER files in the repository.

@OneZero-Y OneZero-Y force-pushed the feat/unit-tests-for-candle-refactoring branch from 830531b to d72f386 Compare September 30, 2025 07:35
Signed-off-by: OneZero-Y <[email protected]>

feat:unit tests for candle refactoring

Signed-off-by: OneZero-Y <[email protected]>

feat:unit tests for candle refactoring

Signed-off-by: OneZero-Y <[email protected]>
@OneZero-Y OneZero-Y force-pushed the feat/unit-tests-for-candle-refactoring branch from d72f386 to da77c32 Compare September 30, 2025 08:09
@rootfs rootfs merged commit a48b0be into vllm-project:feat-candle-refactoring Sep 30, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants